gdk: Make GdkContentFormats immutable

gdk: Add GdkContentFormatsBuilder

Yes, the name is too long.
No, I couldn't think of a shorter one.

gdkdnd: Make GdkDragContext->formats a GdkContentFormats

Instead of it being a GList of GdkAtoms.

contentformats: Rename GtkTargetList

It's now called GdkContentsFormat

gdk: Move GtkTargetList to GDK

It's gonna be renamed next, so put it in the right source file already.

For now retain the old name to keep the diff small.

selection: Make GtkTargetList members private

Outside of the target list, don't access it directly, instead use public
APIs like the newly introduced gtk_target_list_intersects().

dnd: Remove GtkTargetEntry and GtkTargetFlags

warning: We don't do any same-app checks anymore so you currently can
copy local data into external apps.

This will be fixed later.

emoji chooser: break out a helper function

This is just a small cleanup.

remove debug spew

Update POTFILES.in

entry: Add a key binding for the emoji chooser

Make Ctrl-. and Ctrl-; bring up the emoji chooser.

https://bugzilla.gnome.org/show_bug.cgi?id=789160

emoji chooser: Make menu key work as expected

Whenever we have a right-click action, we should make
the menu key work as a keyboard-accessible alternative.

Adwaita: Bring fix focus in flowbox

Bring back the focus rectangle for flowbox children.
Its absence makes keynav in flowboxes really difficult.

emoji chooser: handle right-click

Whereever we handle long-press for touch, it makes sense to handle
right-click as a faster alternative for mouse-based interaction.

This commit makes right-click work to bring up the variation
selector for Emojis.

listbox: Listen to ::stopped on the press gesture

Otherwise, we end up interpreting grab notify's as
button releases, which is not right. This matches
what GtkFlowBox does.

Trivial cleanup

Better to use G_SOURCE_REMOVE than FALSE, for clarity.

emoji chooser: Don't leak gestures

We were not freeing these gestures as we should.

Some more interning

This avoids more strdups at startup.

a11y: Intern a string

cell area: Use GTK_PARAM_READWRITE consistently

wayland: Intern signal names

icon helper: Avoid a few string copies

styleproperty: Use G_PARAM_STATIC_STRINGS

gtk: Intern css names

This avoids a bunch of strdups at startup.

wayland: Intern settings keys

gdk: Intern signal names

We were forgetting this in a few places.

gdk: Use G_PARAM_SPEC_STATIC_STRINGS throughout

This avoids some string copies at startup.

settings: Use gdk_display_set_cursor_theme

This allows us to reduce the amount of ifdef ugliness.

Add private api for setting cursor theme

This is implemented in multiple gdk backends,
and we can avoid and ugly ifdef cascade in gtk
by adding a vfunc for this.

Include gdk-private.h

This gives us access to private gdk api.

gdk: Move code around

Move the gdk_display functions to gdkdisplay.c.
They are misplaced in gdkevents.c.

clipboard: Use gdk_display_get_last_seen_time

This removes the backend dependent code from gtkclipboard.c.
A good thing!

gdk: Add a private method to get a server timestamp

This is needed in the clipboard code. We don't make
it public, since that code is destined to eventually
live in gdk anyway.

Update POTFILES.in

Drop gdkscreen.h

Nothing uses GdkScreen anymore, so we can get rid of it.

quartz: Remove remaining uses of GdkScreen

quartz: Make GdkQuartzScreen a plain object

No need to derive this from GdkScreen anymore.

quartz: Drop an unused include

No GdkScreen apis are used in here.

quartz: Avoid a screen argument

The screen is never used in _gdk_quartz_screen_get_setting,
so drop it.

win32: Remove remaining uses of GdkScreen

win32: Make GdkWin32Screen a plain object

No need to derive this from GdkScreen anymore.

win32: Avoid a screen argument

The screen is never used in _gdk_win32_screen_get_setting,
so drop it.

win32: Remove an unused include

No GdkScreen apis are used in here.

Fix the build

x11: Drop GdkX11Visual as public api

There was no useful functionality left here,
so just remove this from the api altogether.

x11: Stop using GdkScreen altogether

Derive GdkX11Screen directly from GObject, and
remove the last remaining uses of GdkScreen.

x11: Stop using GdkScreen in api

This type is going away, so switch to using GdkX11Screen
everywhere.

Stop calling _gdk_screen_close

It does nothing useful, and is going away.

Stop including gdkscreen.h

It is not used anymore.

broadway: Add missing dependency to build

The generated file clienthtml.h is #included by broadway-server.c, which
is one of the sources of the broadway library — so clienthtml.h needs to
be one of the sources of that library too.

Signed-off-by: Philip Withnall <withnall@endlessm.com>
https://bugzilla.gnome.org/show_bug.cgi?id=790489

broadway: Fix toplevel handling

The toplevels list contains GdkWindowImplBroadways, and we shouldn't
free it.

gdk: Remove gdk_flush()

If you need to flush a display, use gdk_display_flush().

There is never a need to flush displays you don't know about.

gtk-demo: No need to gdk_flush() here

And if there was a need, GDK should have done it.

toolbar: Remove gdk_flush() calls

It's not 2003 anymore.

main: Don't gdk_flush() after every main loop wait

For a start, this makes gtk_main() work different from g_main_loop_run()
calls.

But most importantly, modern GDK does proper syncing itself and doesn't
need to rely on a catch-all to get it right.

Add a comment

Document that gdk-private.h is meant for private gdk<>gtk apis.

Remove gdkprivate.h

This is one-too-many private headers. Move the few
declarations here to gdkinternals.h.

Drop generic error trap api from the docs

Drop gdkmain.h

Stop including gdkmain.h

It is empty now.

Drop an empty file

Nothing deprecated left!

Drop the error trap vfuncs

No longer used.

Drop the generic error trap api

This is only implemented on X11, so we don't
need generic api for it.

tests: Test the x11-specific error trap api

The other one is gone.

x11: Fix error traps in the dnd code

The code for grabbing keys was using a generic
error trap call, and more concerning, it was forgetting
to pop the trap.

selection: Use x11-specific traps

The generic trap api is going away.

mountoperation-x11: Use x11-specific traps

The generic trap api is going away.

Drop an unused variable

These are no longer used.

Drop a duplicate definition

GDK_PRIORITY_EVENTS is already defined in gdkevents.h

Drop gdk_set_program_class

Together with the corresponding getter.
This is x11-specific functionality.

x11: Add gdk_x11_display_set_program_class

The program class is a very X-specific concept.
There should be x11-backend api to deal with it.

wayland: Stop using gdk_get_program_class

This is an abuse, at best. If we need to pass the
app id down here, lets pass the app id.

Remove gdk/gdkx.h

It was just a kludge.

Move some code around

Move gdk_disable_multidevice to gdkdisplaymanager.[hc].

Drop gdk_notify_startup_complete

This function and its variant are just wrappers
around the display-specific function.

gtk: Stop using gdk_notify_startup_complete_with_id

This function is about to go away. Instead, use the
display-specific function that this is a wrapper of.

gdk: Move some headers around

Move gdk_flush and gdk_set_allowed_backends to where they
belong.

Drop gdk_beep

This is an unnecessary wrapper around gdk_display_beep.

im context thai: Stop using gdk_beep

It is about to go away.

Drop gdk_set_double_click_time

This is an unnecessary and not-recommended-to-be-used
wrapper around gdk_display_set_double_click_time.

selection: Remove the info uint

Instead of allowing people to pass a uint user-data, insist on them
comparing mime types.

The user data was a uint instead of a pointer anyway, so uniqueness
could not be guaranteed and it caused more issues than it was worth.
And that's ignoring the fact that it basically wasn't used.

selection: Remove gtk_target_table_*() functions

textview: Stop playing shenanigans with the target list

This breaks GtkSourceView adding custom targets on top of GtkTextView,
but we'll fix that later.

selection: Make GtkTargetEntry not allocatable

And remove it being registered as a boxed type (lol).

HighContrast: misc fixes

The HighContrast theme was not parsing anymore, due to
leftover widget style properties, and some missed cleanups,
like -gtk-icon-effect. Also update for the new focus handling,
and make checks and radios sharp again.

Adwaita: Make checks and radios sharp again

The assets we have are 14x14, set -gtk-icon-size to reflect that.

Change icon-size properties

We no longer support registering custom icon sizes, so
we can make these properties just enums. This also lets
us specify them by nick in ui files. Nice!

image: Update docs

Mention the new icon-size style classes, update the docs
for removed arguments and return values, and add new
functions to the docs.

Document -gtk-icon-size

Add this property to the list of CSS properties.

Support -gtk-icon-size for builtins

This makes css icons more similar to themed icons, which
is the overall goal here.

Drop gtk_icon_size_lookup

Add a new, private gtk_image_get_image_size to replace it, and
update the remaining callers in a suitable way.

recentchooser: Use GIcon instead of GdkPixbuf

iconhelper: Query size via CSS

Instead of looking at the icon size, look at the CSS value for
-gtk-icon-size. Set style classes depending on icon size instead.

Trivially change Adwaita and HighContrast to report the same values as
before.

enums: Change GtkIconSize values

The new values are the ones we intend to keep. The old ones had
duplicated meanings and nobody knew which one to take.

image: Don't take icon size in set_from_definition()

filechooserwidget: icon size is always 16

filechooserbutton: icon size is always 16

So define it that way.

cellrendererspinner: Don't track old icon size

It was only used to avoid gtk_icon_size_lookup() calls and those aren't
expensive.

image: Remove icon-size argument from icon setters

Instead, add a function gtk_image_set_icon_size() for the cases where
overriding the icon size is necessary.

Treat icon sizes the same way as pixel sizes, too. So gtk_image_clear()
no longer unsets the icon size.

image: Change getters

Instead of returning the icon size with them, make
gtk_image_get_icon_name() and gtk_image_get_gicon() only return the icon
itself.

As a benefit, we can turn them into regular getters that return values
instead of requiring out parameters.

Instead, provide gtk_image_get_icon_size() to query the icon size.